Add support for building MPAS on titan with cray and pgi#259
Conversation
|
This was provided as part of https://github.com/MPAS-Dev/MPAS-Release/pull/14 |
There was a problem hiding this comment.
This looks almost identical to the pgi-nersc target.
There was a problem hiding this comment.
Yes, this one does look identical. We can probably remove it from the PR.
|
I think we may be on the verge of a slippery slope when it comes to build targets; if we can determine that the new targets are in fact compatible with existing targets, it might be preferable to use the new targets as synonyms for the existing targets, and to perhaps generalize the existing targets. |
|
I know at least the titan-cray build target is not compatible with previous build targets, or at least the modifications are required for titan at a minimum. I think if the modifications are also compatible with nersc machines, it might be preferable to merge the two together, and just update the modifications, but I don't know if the flags work on nersc machines as well and I'm not sure if there will be time to test this prior to our release. |
douglasjacobsen
force-pushed
the
all/titan_makefile_changes
branch
from
2f8e408 to
185c865
Compare
|
I removed the titan-pgi build target. But I'd still like the titan-cray build target for now. In the future we can look at consolidating these build targets together, but that won't happen prior to v3.0. |
douglasjacobsen
force-pushed
the
all/titan_makefile_changes
branch
from
185c865 to
9f1b947
Compare
2 participants
This adds build targets for PGI and Cray compilers on titan.